Update Spock Tests to run with STRICT_TEST mode by default#11032
Update Spock Tests to run with STRICT_TEST mode by default#11032
STRICT_TEST mode by default#11032Conversation
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 61 metrics, 10 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.61.0-SNAPSHOT~e947d7dd42, baseline=1.61.0-SNAPSHOT~aa7c70f2e7
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.06 s) : 0, 1059709
Total [baseline] (11.176 s) : 0, 11176075
Agent [candidate] (1.068 s) : 0, 1068442
Total [candidate] (11.085 s) : 0, 11084825
section appsec
Agent [baseline] (1.255 s) : 0, 1255268
Total [baseline] (11.18 s) : 0, 11180141
Agent [candidate] (1.262 s) : 0, 1261992
Total [candidate] (11.304 s) : 0, 11303561
section iast
Agent [baseline] (1.228 s) : 0, 1227928
Total [baseline] (11.354 s) : 0, 11354233
Agent [candidate] (1.238 s) : 0, 1238044
Total [candidate] (11.368 s) : 0, 11367604
section profiling
Agent [baseline] (1.191 s) : 0, 1191137
Total [baseline] (10.969 s) : 0, 10969036
Agent [candidate] (1.188 s) : 0, 1187598
Total [candidate] (11.265 s) : 0, 11264769
gantt
title petclinic - break down per module: candidate=1.61.0-SNAPSHOT~e947d7dd42, baseline=1.61.0-SNAPSHOT~aa7c70f2e7
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.234 ms) : 0, 1234
crashtracking [candidate] (1.247 ms) : 0, 1247
BytebuddyAgent [baseline] (633.073 ms) : 0, 633073
BytebuddyAgent [candidate] (641.287 ms) : 0, 641287
AgentMeter [baseline] (29.467 ms) : 0, 29467
AgentMeter [candidate] (29.487 ms) : 0, 29487
GlobalTracer [baseline] (249.352 ms) : 0, 249352
GlobalTracer [candidate] (250.503 ms) : 0, 250503
AppSec [baseline] (32.041 ms) : 0, 32041
AppSec [candidate] (32.171 ms) : 0, 32171
Debugger [baseline] (59.852 ms) : 0, 59852
Debugger [candidate] (60.408 ms) : 0, 60408
Remote Config [baseline] (598.094 µs) : 0, 598
Remote Config [candidate] (607.642 µs) : 0, 608
Telemetry [baseline] (8.119 ms) : 0, 8119
Telemetry [candidate] (8.17 ms) : 0, 8170
Flare Poller [baseline] (9.744 ms) : 0, 9744
Flare Poller [candidate] (8.214 ms) : 0, 8214
section appsec
crashtracking [baseline] (1.239 ms) : 0, 1239
crashtracking [candidate] (1.25 ms) : 0, 1250
BytebuddyAgent [baseline] (665.418 ms) : 0, 665418
BytebuddyAgent [candidate] (672.279 ms) : 0, 672279
AgentMeter [baseline] (12.125 ms) : 0, 12125
AgentMeter [candidate] (12.142 ms) : 0, 12142
GlobalTracer [baseline] (250.731 ms) : 0, 250731
GlobalTracer [candidate] (250.784 ms) : 0, 250784
AppSec [baseline] (185.351 ms) : 0, 185351
AppSec [candidate] (184.911 ms) : 0, 184911
Debugger [baseline] (66.197 ms) : 0, 66197
Debugger [candidate] (66.479 ms) : 0, 66479
Remote Config [baseline] (610.85 µs) : 0, 611
Remote Config [candidate] (608.946 µs) : 0, 609
Telemetry [baseline] (8.708 ms) : 0, 8708
Telemetry [candidate] (8.69 ms) : 0, 8690
Flare Poller [baseline] (3.603 ms) : 0, 3603
Flare Poller [candidate] (3.612 ms) : 0, 3612
IAST [baseline] (24.734 ms) : 0, 24734
IAST [candidate] (24.634 ms) : 0, 24634
section iast
crashtracking [baseline] (1.246 ms) : 0, 1246
crashtracking [candidate] (1.243 ms) : 0, 1243
BytebuddyAgent [baseline] (804.315 ms) : 0, 804315
BytebuddyAgent [candidate] (813.817 ms) : 0, 813817
AgentMeter [baseline] (11.384 ms) : 0, 11384
AgentMeter [candidate] (11.621 ms) : 0, 11621
GlobalTracer [baseline] (239.356 ms) : 0, 239356
GlobalTracer [candidate] (240.474 ms) : 0, 240474
AppSec [baseline] (31.854 ms) : 0, 31854
AppSec [candidate] (32.271 ms) : 0, 32271
Debugger [baseline] (61.476 ms) : 0, 61476
Debugger [candidate] (58.046 ms) : 0, 58046
Remote Config [baseline] (551.916 µs) : 0, 552
Remote Config [candidate] (1.722 ms) : 0, 1722
Telemetry [baseline] (12.212 ms) : 0, 12212
Telemetry [candidate] (12.931 ms) : 0, 12931
Flare Poller [baseline] (3.469 ms) : 0, 3469
Flare Poller [candidate] (3.471 ms) : 0, 3471
IAST [baseline] (25.751 ms) : 0, 25751
IAST [candidate] (25.94 ms) : 0, 25940
section profiling
crashtracking [baseline] (1.195 ms) : 0, 1195
crashtracking [candidate] (1.186 ms) : 0, 1186
BytebuddyAgent [baseline] (696.314 ms) : 0, 696314
BytebuddyAgent [candidate] (693.51 ms) : 0, 693510
AgentMeter [baseline] (9.268 ms) : 0, 9268
AgentMeter [candidate] (9.148 ms) : 0, 9148
GlobalTracer [baseline] (208.092 ms) : 0, 208092
GlobalTracer [candidate] (207.261 ms) : 0, 207261
AppSec [baseline] (32.495 ms) : 0, 32495
AppSec [candidate] (32.609 ms) : 0, 32609
Debugger [baseline] (65.81 ms) : 0, 65810
Debugger [candidate] (65.845 ms) : 0, 65845
Remote Config [baseline] (572.416 µs) : 0, 572
Remote Config [candidate] (581.757 µs) : 0, 582
Telemetry [baseline] (7.802 ms) : 0, 7802
Telemetry [candidate] (7.882 ms) : 0, 7882
Flare Poller [baseline] (3.55 ms) : 0, 3550
Flare Poller [candidate] (3.616 ms) : 0, 3616
ProfilingAgent [baseline] (94.33 ms) : 0, 94330
ProfilingAgent [candidate] (94.485 ms) : 0, 94485
Profiling [baseline] (94.902 ms) : 0, 94902
Profiling [candidate] (95.065 ms) : 0, 95065
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.61.0-SNAPSHOT~e947d7dd42, baseline=1.61.0-SNAPSHOT~aa7c70f2e7
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.063 s) : 0, 1062857
Total [baseline] (8.856 s) : 0, 8856366
Agent [candidate] (1.054 s) : 0, 1053739
Total [candidate] (8.867 s) : 0, 8866856
section iast
Agent [baseline] (1.229 s) : 0, 1229442
Total [baseline] (9.61 s) : 0, 9610004
Agent [candidate] (1.222 s) : 0, 1222295
Total [candidate] (9.561 s) : 0, 9561360
gantt
title insecure-bank - break down per module: candidate=1.61.0-SNAPSHOT~e947d7dd42, baseline=1.61.0-SNAPSHOT~aa7c70f2e7
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.236 ms) : 0, 1236
crashtracking [candidate] (1.225 ms) : 0, 1225
BytebuddyAgent [baseline] (637.804 ms) : 0, 637804
BytebuddyAgent [candidate] (632.796 ms) : 0, 632796
AgentMeter [baseline] (29.595 ms) : 0, 29595
AgentMeter [candidate] (29.25 ms) : 0, 29250
GlobalTracer [baseline] (249.237 ms) : 0, 249237
GlobalTracer [candidate] (248.689 ms) : 0, 248689
AppSec [baseline] (31.868 ms) : 0, 31868
AppSec [candidate] (31.911 ms) : 0, 31911
Debugger [baseline] (59.252 ms) : 0, 59252
Debugger [candidate] (59.214 ms) : 0, 59214
Remote Config [baseline] (597.373 µs) : 0, 597
Remote Config [candidate] (601.7 µs) : 0, 602
Telemetry [baseline] (8.054 ms) : 0, 8054
Telemetry [candidate] (8.033 ms) : 0, 8033
Flare Poller [baseline] (8.911 ms) : 0, 8911
Flare Poller [candidate] (5.82 ms) : 0, 5820
section iast
crashtracking [baseline] (1.244 ms) : 0, 1244
crashtracking [candidate] (1.224 ms) : 0, 1224
BytebuddyAgent [baseline] (806.18 ms) : 0, 806180
BytebuddyAgent [candidate] (800.816 ms) : 0, 800816
AgentMeter [baseline] (11.552 ms) : 0, 11552
AgentMeter [candidate] (11.376 ms) : 0, 11376
GlobalTracer [baseline] (239.414 ms) : 0, 239414
GlobalTracer [candidate] (239.02 ms) : 0, 239020
AppSec [baseline] (30.3 ms) : 0, 30300
AppSec [candidate] (33.338 ms) : 0, 33338
Debugger [baseline] (61.026 ms) : 0, 61026
Debugger [candidate] (58.258 ms) : 0, 58258
Remote Config [baseline] (1.168 ms) : 0, 1168
Remote Config [candidate] (1.11 ms) : 0, 1110
Telemetry [baseline] (12.942 ms) : 0, 12942
Telemetry [candidate] (11.559 ms) : 0, 11559
Flare Poller [baseline] (3.463 ms) : 0, 3463
Flare Poller [candidate] (3.461 ms) : 0, 3461
IAST [baseline] (25.888 ms) : 0, 25888
IAST [candidate] (25.823 ms) : 0, 25823
LoadParameters
See matching parameters
SummaryFound 3 performance improvements and 0 performance regressions! Performance is the same for 17 metrics, 16 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.61.0-SNAPSHOT~e947d7dd42, baseline=1.61.0-SNAPSHOT~aa7c70f2e7
dateFormat X
axisFormat %s
section baseline
no_agent (1.258 ms) : 1245, 1271
. : milestone, 1258,
iast (3.394 ms) : 3345, 3443
. : milestone, 3394,
iast_FULL (6.377 ms) : 6311, 6444
. : milestone, 6377,
iast_GLOBAL (3.717 ms) : 3652, 3781
. : milestone, 3717,
profiling (2.224 ms) : 2202, 2246
. : milestone, 2224,
tracing (1.866 ms) : 1850, 1881
. : milestone, 1866,
section candidate
no_agent (1.246 ms) : 1234, 1258
. : milestone, 1246,
iast (3.215 ms) : 3171, 3258
. : milestone, 3215,
iast_FULL (5.985 ms) : 5925, 6045
. : milestone, 5985,
iast_GLOBAL (3.672 ms) : 3608, 3736
. : milestone, 3672,
profiling (2.179 ms) : 2158, 2199
. : milestone, 2179,
tracing (1.945 ms) : 1928, 1963
. : milestone, 1945,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.61.0-SNAPSHOT~e947d7dd42, baseline=1.61.0-SNAPSHOT~aa7c70f2e7
dateFormat X
axisFormat %s
section baseline
no_agent (18.17 ms) : 17985, 18354
. : milestone, 18170,
appsec (19.057 ms) : 18866, 19249
. : milestone, 19057,
code_origins (18.189 ms) : 18012, 18365
. : milestone, 18189,
iast (18.16 ms) : 17980, 18340
. : milestone, 18160,
profiling (18.486 ms) : 18301, 18671
. : milestone, 18486,
tracing (18.167 ms) : 17986, 18347
. : milestone, 18167,
section candidate
no_agent (19.382 ms) : 19187, 19576
. : milestone, 19382,
appsec (18.868 ms) : 18679, 19058
. : milestone, 18868,
code_origins (17.865 ms) : 17688, 18042
. : milestone, 17865,
iast (18.021 ms) : 17843, 18199
. : milestone, 18021,
profiling (18.424 ms) : 18241, 18607
. : milestone, 18424,
tracing (17.644 ms) : 17469, 17819
. : milestone, 17644,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 1 unstable metrics. Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.61.0-SNAPSHOT~e947d7dd42, baseline=1.61.0-SNAPSHOT~aa7c70f2e7
dateFormat X
axisFormat %s
section baseline
no_agent (15.562 s) : 15562000, 15562000
. : milestone, 15562000,
appsec (14.901 s) : 14901000, 14901000
. : milestone, 14901000,
iast (18.271 s) : 18271000, 18271000
. : milestone, 18271000,
iast_GLOBAL (17.9 s) : 17900000, 17900000
. : milestone, 17900000,
profiling (15.057 s) : 15057000, 15057000
. : milestone, 15057000,
tracing (14.935 s) : 14935000, 14935000
. : milestone, 14935000,
section candidate
no_agent (15.178 s) : 15178000, 15178000
. : milestone, 15178000,
appsec (14.913 s) : 14913000, 14913000
. : milestone, 14913000,
iast (18.26 s) : 18260000, 18260000
. : milestone, 18260000,
iast_GLOBAL (18.061 s) : 18061000, 18061000
. : milestone, 18061000,
profiling (15.115 s) : 15115000, 15115000
. : milestone, 15115000,
tracing (14.924 s) : 14924000, 14924000
. : milestone, 14924000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.61.0-SNAPSHOT~e947d7dd42, baseline=1.61.0-SNAPSHOT~aa7c70f2e7
dateFormat X
axisFormat %s
section baseline
no_agent (1.493 ms) : 1482, 1505
. : milestone, 1493,
appsec (3.852 ms) : 3628, 4076
. : milestone, 3852,
iast (2.297 ms) : 2228, 2367
. : milestone, 2297,
iast_GLOBAL (2.336 ms) : 2265, 2406
. : milestone, 2336,
profiling (2.11 ms) : 2055, 2165
. : milestone, 2110,
tracing (2.101 ms) : 2046, 2155
. : milestone, 2101,
section candidate
no_agent (1.495 ms) : 1483, 1507
. : milestone, 1495,
appsec (3.877 ms) : 3653, 4101
. : milestone, 3877,
iast (2.287 ms) : 2217, 2356
. : milestone, 2287,
iast_GLOBAL (2.338 ms) : 2268, 2409
. : milestone, 2338,
profiling (2.135 ms) : 2079, 2191
. : milestone, 2135,
tracing (2.095 ms) : 2041, 2150
. : milestone, 2095,
|
5ba6f69 to
b0147c9
Compare
95b7e3f to
82a89e0
Compare
|
@codex review |
|
ok so most of the changes are coming from the test files where we override that global config |
|
Codex Review: Didn't find any major issues. What shall we delve into next? ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
The merge request has been interrupted because the build 106438802 took longer than expected. The current limit for the base branch 'master' is 120 minutes. |
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
The merge request has been interrupted because the build 106484564 took longer than expected. The current limit for the base branch 'master' is 120 minutes. |
What Does This Do
This PR updates Spock tests to run with
STRICT_TESTby default. This means that all Environment Variables that are queried but not documented will fail the test. We fail the test by throwing an error when querying such environment variable, but if the test has automatic error handling we also throw an error during test cleanup to log the missing configurations fromsupported-configurations.json.If a test is using fake configurations, the test itself can override the strictness mode with
TESTto bypass this mechanism. Note that this is being done w/ many Instrumentation tests as they create fake decorators that register fake configs.Motivation
Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any other useful labelsclose,fix, or any linking keywords when referencing an issueUse
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]
Note: Once your PR is ready to merge, add it to the merge queue by commenting
/merge./merge -ccancels the queue request./merge -f --reason "reason"skips all merge queue checks; please use this judiciously, as some checks do not run at the PR-level. For more information, see this doc.